fix(KFLUXBUGS-1581): force releaseNotes.type when cves defined #619
Conversation
|
Skipping CI for Draft Pull Request. |
johnbieren
force-pushed
the
kfluxbugs1581
branch
2 times, most recently
from
30d70b4
to
9b3c6ae
Compare
|
@johnbieren: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
| NUM_CVES=$(jq '.content.images[]?.cves.fixed // 0 | length' <<< "$advisoryData" \ | ||
| | awk '{sum=sum+$0} END{print sum}') |
There was a problem hiding this comment.
You can do the sum directly with jq. Also, while it works with the default 0, you're expecting an array, so defaulting to [] seems more correct.
| NUM_CVES=$(jq '.content.images[]?.cves.fixed // 0 | length' <<< "$advisoryData" \ | |
| | awk '{sum=sum+$0} END{print sum}') | |
| NUM_CVES=$(jq '[.content.images[]?.cves.fixed // [] | length] | add' <<< "$advisoryData") |
But if you prefer awk, that's fine with me.
There was a problem hiding this comment.
Much better, thanks for the suggestion
This commit modifies the create-advisory task to fail when releaseNotes.type is RHSA but there are no CVEs listed as fixed. Also, if there are CVEs fixed, the type is forced to be RHSA regardless of what type the user provides. Signed-off-by: Johnny Bieren <[email protected]>
johnbieren
force-pushed
the
kfluxbugs1581
branch
from
9b3c6ae
to
835e219
Compare
This commit modifies the create-advisory task to fail when releaseNotes.type is RHSA but there are no CVEs listed as fixed. Also, if there are CVEs fixed, the type is forced to be RHSA regardless of what type the user provides.